Skip to content

fix: respect OTEL_EXPORTER_OTLP_ENDPOINT when setting trace/log/metrics endpoints#732

Merged
liustve merged 2 commits intomainfrom
fix/respect-base-otlp-endpoint
Apr 23, 2026
Merged

fix: respect OTEL_EXPORTER_OTLP_ENDPOINT when setting trace/log/metrics endpoints#732
liustve merged 2 commits intomainfrom
fix/respect-base-otlp-endpoint

Conversation

@liustve
Copy link
Copy Markdown
Contributor

@liustve liustve commented Apr 23, 2026

Description of changes:

…ints

When OTEL_EXPORTER_OTLP_ENDPOINT is set, skip defaulting the specific
trace and log endpoint vars so the base URL is honored by the OTel SDK.
This was a regression from #729 which dropped the guard.
@liustve liustve requested a review from a team as a code owner April 23, 2026 19:33
@liustve liustve changed the title fix: respect OTEL_EXPORTER_OTLP_ENDPOINT when setting trace/log endpoints fix: respect OTEL_EXPORTER_OTLP_ENDPOINT when setting trace/log/metrics endpoints Apr 23, 2026
@liustve liustve added the skip changelog doesn't need a CHANGELOG entry label Apr 23, 2026
@liustve liustve enabled auto-merge (squash) April 23, 2026 20:23
@liustve liustve merged commit 9aa2903 into main Apr 23, 2026
28 of 30 checks passed
@liustve liustve deleted the fix/respect-base-otlp-endpoint branch April 23, 2026 20:23
liustve added a commit to aws-observability/aws-otel-js-instrumentation that referenced this pull request Apr 24, 2026
*Description of changes*
- When `OTEL_EXPORTER_OTLP_ENDPOINT` is set, skip setting
signal-specific endpoints derives them from the base URL
- Aligns with the Python distro behavior
(aws-observability/aws-otel-python-instrumentation#732)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip changelog doesn't need a CHANGELOG entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants